Skip to content

Conversation

@nrichers
Copy link
Collaborator

@nrichers nrichers commented Sep 25, 2025

Pull Request Description

What and why?

This PR applies only the following changes from #956:

  • Fix links to Python API and Public REST API in footer & dropdown
  • Remove icons and text labels from dropdown
  • Fix link color and remove underline

Relates to sc-12329

Capto_ 2025-09-25_03-55-24_pm Capto_ 2025-09-25_04-18-27_pm

To get this PR here into prod safely, I will cherry pick the merge commit only.

How to test

What needs special review?

Dependencies, breaking changes, and deployment notes

Release notes

Checklist

  • What and why
  • Screenshots or videos (Frontend)
  • How to test
  • What needs special review
  • Dependencies, breaking changes, and deployment notes
  • Labels applied
  • PR linked to Shortcut
  • Unit tests added (Backend)
  • Tested locally
  • Documentation updated (if required)
  • Environment variable additions/changes documented (if required)

@nrichers nrichers added the internal Not to be externalized in the release notes label Sep 25, 2025
@nrichers nrichers requested a review from validbeck September 25, 2025 23:25
Copy link
Collaborator

@validbeck validbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this case I think we should actually retain the link changes from #949. I like the no-underline at all even on hover better personally, but it's just not very accessible.

@nrichers
Copy link
Collaborator Author

nrichers commented Oct 6, 2025

In this case I think we should actually retain the link changes from #949.

@validbeck I ended up adding some very specific SCSS to underline only specific types of links.

Capto_.2025-10-06_12-58-22_pm.mov

@nrichers nrichers requested a review from validbeck October 6, 2025 19:59
Copy link
Collaborator

@validbeck validbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, you're going to hate me... but...

  1. Both the "developer" and "training" portals are still missing the underline on link hover. 🫠
  2. The drop-down menu now looks a bit off with the icon missing, in terms of padding/spacing in comparison to how wide the box is. My suggestion:
.navbar-nav .dropdown-menu .dropdown-text {
  padding-left: 10px;
}

Results:

OLD NEW
old new

@nrichers
Copy link
Collaborator Author

nrichers commented Oct 7, 2025

  • Both the "developer" and "training" portals are still missing the underline on link hover. 🫠

Ah, the 'we have styles in many places' issue. 🙃 Can't wait until we define all of this stuff once and then simply import wherever needed ...

The developer portal links underline for me without additional changes:

Capto_ 2025-10-07_09-18-13_am Capto_ 2025-10-07_09-18-53_am Capto_ 2025-10-07_09-19-21_am

The training portal links I could find for the targetted classes work, for example:

Capto_ 2025-10-07_09-20-26_am

I did add some temporary CSS fixes for two additional classes:

Capto_ 2025-10-07_09-22-00_am Capto_ 2025-10-07_09-22-28_am
  • The drop-down menu now looks a bit off with the icon missing, in terms of padding/spacing in

Added as suggested but also noticed the bottom then looked squished, so gave it a bit more padding:

Capto_ 2025-10-07_08-54-37_am

Note navbar stuff gets split out into a separate SCSS file in my other PR, so I really don't want to do more here than address your comment. Also note !important was required here or the bottom padding wouldn't work. 🤷

(Also, please don't ask me to fix additional CSS in this PR? 😬 The purpose of this PR was to add the API links. We need to complete the conversion to modular SCSS and then start building up our styles correctly again.)

@validbeck
Copy link
Collaborator

The developer and training portal links still don't work for me in your latest preview for the notebooks. Here's a video:

Screen.Recording.2025-10-07.at.9.55.43.AM.mov

Also, please don't ask me to fix additional CSS in this PR? 😬 The purpose of this PR was to add the API links. We need to complete the conversion to modular SCSS and then start building up our styles correctly again.)

Fair enough for the drop-down menu, I was just pointing out the padding looks not-too-great, hence me suggesting it in a comment rather than pushing a commit.

@nrichers
Copy link
Collaborator Author

nrichers commented Oct 7, 2025

The developer and training portal links still don't work for me in your latest preview for the notebooks.

That is ... odd. Not sure what to say, but let's compare notes at our standup today? I might suggest we defer whatever fix might be needed, as this is more of a nice-to have and Mehdi recently asked me about the API reference links again ...

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

PR Summary

This PR introduces several user interface improvements and styling updates to the website:

  1. Navigation Menu Updates:

    • Revised navigation text by removing icon markdown, simplifying labels like "About", "Get Started", "Guides", "Support", and "Releases".
    • Added new navigation items for "Python API" and "Public REST API", and updated the corresponding links in the menu and footer sections.
  2. Styling Enhancements:

    • Updated link styles in multiple SCSS and CSS files. The link hover behavior has been standardized with a solid underline effect and uses the updated color #083E44 with an !important override.
    • Adjustments in theme settings to use a new link color (#DE257E) instead of inheriting from the body color.
    • Increased padding in some UI elements to improve layout consistency.
  3. .gitignore Update:

    • Added a pattern to ignore all files with the .quarto_ipynb extension across the repository, ensuring they don’t get inadvertently tracked.

Overall, the changes aim to improve the visual consistency of the site’s navigation and link interactions while ensuring non-irrelevant files are properly ignored during version control.

Test Suggestions

  • Verify that the navigation menu displays the correct text labels without icon markdown and that the links (e.g., 'Python API' and 'Public REST API') correctly route to the intended pages.
  • Test the hover behavior on various links to ensure the underline styles are applied consistently across the site and in specific sections as intended.
  • Inspect layout changes (e.g., adjusted padding in site/styles.css) to confirm that the UI renders properly on different devices and screen sizes.
  • Confirm that .gitignore properly excludes files matching the '**/*.quarto_ipynb' pattern from version control.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

Vale source linter

⚠️ WARN: Found 14257 issues across 690 files — Check the workflow run or download the full report

Show first 30 files with issues

site/404.qmd

  • Line 2: Use parentheses judiciously. (suggestion)
  • Line 3: Did you really mean 'html'? (error)
  • Line 53: Did you really mean 'const'? (error)
  • Line 53: Use parentheses judiciously. (suggestion)
  • Line 54: Did you really mean 'const'? (error)
  • Line 54: Did you really mean 'ctx'? (error)
  • Line 54: Use parentheses judiciously. (suggestion)
  • Line 54: Put a nonbreaking space between the number and the unit in '2d'. (error)
  • Line 58: Did you really mean 'const'? (error)
  • Line 58: Did you really mean 'fontSize'? (error)
  • ... and 23 more issues

site/about/contributing/style-guide/conventions.qmd

  • Line 6: Use 'it's' instead of 'it is'. (suggestion)
  • Line 6: In general, use active voice instead of passive voice ('is received'). (suggestion)
  • Line 6: Use parentheses judiciously. (suggestion)
  • Line 9: Use 'capability' or 'feature' instead of 'functionality'. (warning)
  • Line 9: Use 'shouldn't' instead of 'should not'. (suggestion)
  • Line 9: In general, use active voice instead of passive voice ('be lost'). (suggestion)
  • Line 15: Did you really mean 'Palo'? (error)
  • Line 19: Avoid using 'will'. (warning)
  • Line 19: In general, use American spelling instead of 'centre'. (warning)
  • Line 19: Avoid using 'will'. (warning)
  • ... and 180 more issues

site/about/contributing/style-guide/style-guide.qmd

  • Line 2: Spell out 'USING', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'TITLE', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 22: Try to avoid using first-person plural like 'we'. (warning)
  • Line 22: Try to avoid using first-person plural like 'we'. (warning)
  • Line 22: Try to avoid using first-person plural like 'We'. (warning)
  • Line 24: Try to avoid using first-person plural like 'Our'. (warning)
  • Line 24: Try to avoid using first-person plural like 'our'. (warning)
  • ... and 13 more issues

site/about/contributing/style-guide/voice-and-tone.qmd

  • Line 6: Try to avoid using first-person plural like 'we'. (warning)
  • Line 8: Try to avoid using first-person plural like 'we'. (warning)
  • Line 8: Try to avoid using first-person plural like 'our'. (warning)
  • Line 8: Try to avoid using first-person plural like 'we'. (warning)
  • Line 8: Try to avoid using first-person plural like 'we'. (warning)
  • Line 18: Avoid using 'will'. (warning)
  • Line 18: In general, use active voice instead of passive voice ('be reviewed'). (suggestion)
  • Line 22: Use parentheses judiciously. (suggestion)
  • Line 22: Try to avoid using first-person plural like 'we'. (warning)
  • Line 22: Don't put a space before or after a dash. (error)
  • ... and 39 more issues

site/about/contributing/validmind-community.qmd

  • Line 8: Use parentheses judiciously. (suggestion)
  • Line 8: Spell out 'MRM', if it's unfamiliar to the audience. (suggestion)
  • Line 8: Try to avoid using first-person plural like 'our'. (warning)
  • Line 8: Don't use exclamation points in text. (error)
  • Line 8: Try to avoid using first-person plural like 'us'. (warning)
  • Line 14: Try to avoid using first-person plural like 'our'. (warning)
  • Line 19: Don't use exclamation points in text. (error)
  • Line 25: Use parentheses judiciously. (suggestion)
  • Line 31: Try to avoid using first-person plural like 'our'. (warning)
  • Line 42: Use parentheses judiciously. (suggestion)
  • ... and 37 more issues

site/about/deployment/_deployment-available-options.qmd

  • Line 1: Try to avoid using first-person plural like 'Our'. (warning)
  • Line 1: Use parentheses judiciously. (suggestion)
  • Line 1: Use 'Google Cloud Platform' or 'GCP' instead of 'Cloud'. (warning)
  • Line 3: Try to avoid using first-person plural like 'We'. (warning)
  • Line 5: Don't put a space before or after a dash. (error)
  • Line 5: Use parentheses judiciously. (suggestion)
  • Line 5: In general, use active voice instead of passive voice ('be established'). (suggestion)
  • Line 7: Use parentheses judiciously. (suggestion)
  • Line 7: Spell out 'VPV', if it's unfamiliar to the audience. (suggestion)
  • Line 7: Don't put a space before or after a dash. (error)
  • ... and 1 more issues

site/about/deployment/deployment-options.qmd

  • Line 24: Use parentheses judiciously. (suggestion)
  • Line 26: Use 'Google Cloud Platform' or 'GCP' instead of 'Cloud'. (warning)
  • Line 28: Try to avoid using first-person plural like 'our'. (warning)
  • Line 32: Don't put a space before or after a dash. (error)
  • Line 32: Did you really mean 'validators'? (error)
  • Line 34: Don't put a space before or after a dash. (error)
  • Line 36: Don't put a space before or after a dash. (error)
  • Line 38: Don't put a space before or after a dash. (error)
  • Line 42: Try to avoid using first-person plural like 'We'. (warning)
  • Line 42: Use 'data is' instead of 'data are'. (warning)
  • ... and 12 more issues

site/about/deployment/multi-tenant-cloud.qmd

  • Line 13: Use 'tap' instead of 'touch'. (warning)
  • Line 17: Did you really mean 'deployment_'? (error)
  • Line 19: Use 'it's' instead of 'it is'. (suggestion)
  • Line 21: Use parentheses judiciously. (suggestion)
  • Line 29: Don't put a space before or after a dash. (error)
  • Line 30: Don't put a space before or after a dash. (error)
  • Line 31: Don't put a space before or after a dash. (error)
  • Line 32: Don't put a space before or after a dash. (error)
  • Line 39: In general, use active voice instead of passive voice ('is encrypted'). (suggestion)
  • Line 40: Use parentheses judiciously. (suggestion)
  • ... and 2 more issues

site/about/deployment/virtual-private-validmind.qmd

  • Line 2: Spell out 'USING', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'TITLE', if it's unfamiliar to the audience. (suggestion)
  • Line 2: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 3: Use parentheses judiciously. (suggestion)
  • Line 3: Spell out 'VPV', if it's unfamiliar to the audience. (suggestion)
  • Line 14: Use 'tap' instead of 'touch'. (warning)
  • Line 18: Use parentheses judiciously. (suggestion)
  • Line 18: Spell out 'VPV', if it's unfamiliar to the audience. (suggestion)
  • ... and 8 more issues

site/about/fine-print/data-privacy-policy.qmd

  • Line 12: Try to avoid using first-person plural like 'our'. (warning)
  • Line 19: Try to avoid using first-person plural like 'we'. (warning)
  • Line 21: Try to avoid using first-person plural like 'Our'. (warning)
  • Line 21: Try to avoid using first-person plural like 'our'. (warning)
  • Line 21: Try to avoid using first-person plural like 'our'. (warning)
  • Line 26: Try to avoid using first-person plural like 'our'. (warning)
  • Line 26: Try to avoid using first-person plural like 'we'. (warning)
  • Line 26: Try to avoid using first-person plural like 'our'. (warning)
  • Line 34: Use 'what's' instead of 'What is'. (suggestion)
  • Line 36: Try to avoid using first-person plural like 'our'. (warning)
  • ... and 49 more issues

site/about/fine-print/license-agreement.qmd

  • Line 9: Spell out 'READ', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'THIS', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Use parentheses judiciously. (suggestion)
  • Line 9: Spell out 'THIS', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'USING', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'USING', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'THE', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'YOU', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'ARE', if it's unfamiliar to the audience. (suggestion)
  • ... and 115 more issues

site/about/glossary/_ai.qmd

  • Line 1: Spell out 'IBM', if it's unfamiliar to the audience. (suggestion)
  • Line 1: Use parentheses judiciously. (suggestion)
  • Line 3: Use parentheses judiciously. (suggestion)
  • Line 7: Use parentheses judiciously. (suggestion)
  • Line 9: Use parentheses judiciously. (suggestion)
  • Line 10: In general, use active voice instead of passive voice ('were trained'). (suggestion)
  • Line 12: Use parentheses judiciously. (suggestion)
  • Line 12: Spell out 'LLM', if it's unfamiliar to the audience. (suggestion)
  • Line 13: Use parentheses judiciously. (suggestion)

site/about/glossary/_attestation.qmd

  • Line 2: Did you really mean 'validator'? (error)
  • Line 2: In general, use active voice instead of passive voice ('are met'). (suggestion)
  • Line 5: In general, use active voice instead of passive voice ('is triggered'). (suggestion)
  • Line 14: Did you really mean 'validators'? (error)

site/about/glossary/_developer-tools.qmd

  • Line 2: Use parentheses judiciously. (suggestion)
  • Line 2: Use 'capability' or 'feature' instead of 'functionality'. (warning)
  • Line 15: Use 'that's' instead of 'that is'. (suggestion)
  • Line 20: Try to avoid using first-person plural like 'our'. (warning)
  • Line 30: Use parentheses judiciously. (suggestion)
  • Line 32: Use parentheses judiciously. (suggestion)

site/about/glossary/_model-documentation.qmd

  • Line 1: Spell out 'MODEL', if it's unfamiliar to the audience. (suggestion)
  • Line 1: Spell out 'NEEDS', if it's unfamiliar to the audience. (suggestion)
  • Line 1: In general, use active voice instead of passive voice ('BE ADDED'). (suggestion)
  • Line 1: Spell out 'ADDED', if it's unfamiliar to the audience. (suggestion)
  • Line 3: Use parentheses judiciously. (suggestion)
  • Line 4: Spell out 'ABC', if it's unfamiliar to the audience. (suggestion)

site/about/glossary/_models.qmd

  • Line 4: Use parentheses judiciously. (suggestion)
  • Line 7: In general, use active voice instead of passive voice ('are derived'). (suggestion)
  • Line 7: In general, use active voice instead of passive voice ('is achieved'). (suggestion)
  • Line 11: Use parentheses judiciously. (suggestion)
  • Line 15: Use 'it's' instead of 'it is'. (suggestion)
  • Line 15: In general, use active voice instead of passive voice ('is decommissioned'). (suggestion)
  • Line 15: Use parentheses judiciously. (suggestion)
  • Line 22: Use parentheses judiciously. (suggestion)
  • Line 22: Spell out 'MRM', if it's unfamiliar to the audience. (suggestion)
  • Line 23: In general, use active voice instead of passive voice ('are developed'). (suggestion)

site/about/glossary/_monitoring.qmd

  • Line 1: Spell out 'NEEDS', if it's unfamiliar to the audience. (suggestion)
  • Line 1: In general, use active voice instead of passive voice ('BE ADDED'). (suggestion)
  • Line 1: Spell out 'ADDED', if it's unfamiliar to the audience. (suggestion)
  • Line 3: Use parentheses judiciously. (suggestion)
  • Line 4: Spell out 'ABC', if it's unfamiliar to the audience. (suggestion)

site/about/glossary/_mrm.qmd

  • Line 3: Put a nonbreaking space between the number and the unit in '1s'. (error)
  • Line 3: Spell out all ordinal numbers ('1st') in text. (error)
  • Line 4: Don't use plurals in parentheses such as in 'unit(s)'. (error)
  • Line 4: Use parentheses judiciously. (suggestion)
  • Line 4: Spell out all ordinal numbers ('1st') in text. (error)
  • Line 4: Put a nonbreaking space between the number and the unit in '1s'. (error)
  • Line 4: Use 'they're' instead of 'they are'. (suggestion)
  • Line 6: Spell out all ordinal numbers ('2nd') in text. (error)
  • Line 7: Spell out all ordinal numbers ('2nd') in text. (error)
  • Line 7: Did you really mean 'validators'? (error)
  • ... and 11 more issues

site/about/glossary/_validation-reports.qmd

  • Line 1: Spell out 'NEEDS', if it's unfamiliar to the audience. (suggestion)
  • Line 1: In general, use active voice instead of passive voice ('BE ADDED'). (suggestion)
  • Line 1: Spell out 'ADDED', if it's unfamiliar to the audience. (suggestion)
  • Line 3: Use parentheses judiciously. (suggestion)
  • Line 4: Spell out 'ABC', if it's unfamiliar to the audience. (suggestion)

site/about/glossary/_validmind-features.qmd

  • Line 7: In general, use active voice instead of passive voice ('are used'). (suggestion)
  • Line 7: Use parentheses judiciously. (suggestion)

site/about/glossary/_validmind.qmd

  • Line 3: In general, use active voice instead of passive voice ('are intertwined'). (suggestion)
  • Line 5: Use parentheses judiciously. (suggestion)
  • Line 6: Use 'open source' instead of 'open-source'. (warning)
  • Line 6: Use parentheses judiciously. (suggestion)
  • Line 6: Did you really mean 'overfit'? (error)
  • Line 8: Use parentheses judiciously. (suggestion)
  • Line 9: Use parentheses judiciously. (suggestion)
  • Line 9: Did you really mean 'APIs'? (error)

site/about/glossary/glossary.qmd

  • Line 7: Spell out 'NEW', if it's unfamiliar to the audience. (suggestion)
  • Line 7: Spell out 'TERM', if it's unfamiliar to the audience. (suggestion)
  • Line 7: Spell out 'NEEDS', if it's unfamiliar to the audience. (suggestion)
  • Line 7: In general, use active voice instead of passive voice ('BE ADDED'). (suggestion)
  • Line 7: Spell out 'ADDED', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Spell out 'ABC', if it's unfamiliar to the audience. (suggestion)
  • Line 9: Use parentheses judiciously. (suggestion)
  • Line 9: Use 'for example' instead of 'e.g.'. (error)
  • Line 10: Spell out 'KEY', if it's unfamiliar to the audience. (suggestion)
  • Line 10: Spell out 'TERMS', if it's unfamiliar to the audience. (suggestion)
  • ... and 9 more issues

site/about/glossary/key_concepts/_docs.qmd

  • Line 4: Use 'app' instead of 'application'. (warning)

site/about/glossary/key_concepts/_inputs.qmd

  • Line 2: In general, use active voice instead of passive voice ('be evaluated'). (suggestion)
  • Line 4: In general, use active voice instead of passive voice ('been initialized'). (suggestion)
  • Line 4: Use parentheses judiciously. (suggestion)
  • Line 5: In general, use active voice instead of passive voice ('been initialized'). (suggestion)
  • Line 5: Use parentheses judiciously. (suggestion)
  • Line 6: In general, use active voice instead of passive voice ('is used'). (suggestion)
  • Line 7: In general, use active voice instead of passive voice ('is used'). (suggestion)
  • Line 7: Use parentheses judiciously. (suggestion)

site/about/glossary/key_concepts/_key-concepts.qmd

  • Line 1: Spell out 'KEY', if it's unfamiliar to the audience. (suggestion)
  • Line 1: Spell out 'NEEDS', if it's unfamiliar to the audience. (suggestion)
  • Line 1: In general, use active voice instead of passive voice ('BE ADDED'). (suggestion)
  • Line 1: Spell out 'ADDED', if it's unfamiliar to the audience. (suggestion)
  • Line 3: Use parentheses judiciously. (suggestion)
  • Line 5: Use parentheses judiciously. (suggestion)
  • Line 5: Use 'for example' instead of 'e.g.'. (error)
  • Line 5: Spell out 'ABC', if it's unfamiliar to the audience. (suggestion)

site/about/glossary/key_concepts/_metrics.qmd

  • Line 2: Use 'don't' instead of 'do not'. (suggestion)
  • Line 2: In general, use active voice instead of passive voice ('be registered'). (suggestion)
  • Line 2: In general, use active voice instead of passive voice ('be used'). (suggestion)
  • Line 4: In general, use active voice instead of passive voice ('be thought'). (suggestion)

site/about/glossary/key_concepts/_monitoring-template.qmd

  • Line 2: In general, use active voice instead of passive voice ('be tracked'). (suggestion)
  • Line 2: In general, use active voice instead of passive voice ('be conducted'). (suggestion)
  • Line 2: In general, use active voice instead of passive voice ('be displayed'). (suggestion)
  • Line 4: Did you really mean 'boilerplates'? (error)

site/about/glossary/key_concepts/_outputs.qmd

  • Line 2: Use parentheses judiciously. (suggestion)
  • Line 2: 's.D' should have one space. (error)

site/about/glossary/key_concepts/_parameters.qmd

  • Line 2: In general, use active voice instead of passive voice ('be passed'). (suggestion)

site/about/glossary/key_concepts/_template.qmd

  • Line 2: In general, use active voice instead of passive voice ('be run'). (suggestion)
  • Line 2: In general, use active voice instead of passive voice ('be displayed'). (suggestion)
  • Line 5: Did you really mean 'validators'? (error)
  • Line 7: Did you really mean 'boilerplates'? (error)

... and 660 more files with issues

@nrichers
Copy link
Collaborator Author

nrichers commented Oct 7, 2025

@validbeck final commit on this PR from me, this one

  • fixes the issue with underlines on links that are inside of
    • prevents underlines on listing cards, required because some of our what's next links are basic links and some are cards.

    If this is not acceptable, then I will close this PR - most of the changes I've made over the last couple of days are stop-gap measures that should go away once we sort out more of our SCSS correctly.

    (For example, it would be much simpler to specify a single rule on underlines for links that includes a directive about which which links we DON'T want to underline globally. But this PR doesn't include some of the improvements from the other PR that are required to implement this cleanly and there's no point retrofitting them if the work's already done elsewhere.)

@nrichers nrichers requested a review from validbeck October 7, 2025 23:42
@github-actions
Copy link
Contributor

github-actions bot commented Oct 7, 2025

Validate docs site

✓ INFO: A live preview of the docs site is available — Open the preview

Copy link
Collaborator

@validbeck validbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good enough — most of the major issues are resolved in this. Can we make a note that the underline on hover still doesn't work on notebooks and to take a look at that as we continue to clean up our CSS?

@nrichers
Copy link
Collaborator Author

nrichers commented Oct 8, 2025

Can we make a note that the underline on hover still doesn't work on notebooks and to take a look at that as we continue to clean up our CSS?

Added as a comment to #956. As mentioned, I will continue to work on that PR on the side as it contains other fixes we still need.

@nrichers nrichers merged commit cd02ef6 into main Oct 8, 2025
6 of 7 checks passed
@nrichers nrichers deleted the nrichers/sc-12329/add-public-api-to-footer-2 branch October 8, 2025 21:22
nrichers added a commit that referenced this pull request Oct 10, 2025
* Removing validmind version variable (#921)

* infra: Merge main into staging PRs created by staging deploy now indicate original PR# (#924)

* Updating main into staging workflow

* Testing branch versions

* Removing test file

* Fix Makefile help

* infra: Delete older PR preview comments from validate & execute flows (#932)

* Adding deletion of old preview comments back in

* Removing unused variable

* Adding same thing to notebooks

* Edit to notebook, should remove old comment

* hm

* Deleting test execution notebook

* ValidMind Platform releases: Exclude `admin-ui` from roundup listing page (#933)

* Exclude admin-ui from platform releases roundup

* Adjust

* Remove old variable usage

---------

Co-authored-by: Nik Richers <nik@validmind.ai>

* docs: Export findings (#939)

* Setup: Customize model findings layout

* Save point

* Save point

* Draft: Customize model finding layouts

* Setup: Export findings

* Draft: Export findings

* Export documentation > Export documents

* Editing

* Cleanup

* Proofread

* Filter

* Update API reference to use prod server (#938)

* Update API reference to use prod server

* Update Makefile for admin API as well

* docs: Export inventory (#943)

* Setup: Export inventory

* Draft: Export inventory

* Proofreading

* Oops

* docs: Updated Export findings (#944)

* Setup

* Proofread

* Testing pinned Quarto version

* docs: Customize Virtual Document Validator (#947)

* Set up: Manage regulations and policies

* WIP

* Customize VDV draft

* Edits: Check documentation for compliance

* Proofread

* Quick UI updates

* Pin a Quarto version specified in Action variables for our deployment workflows (#951)

* Testing pinned Quarto version

* Applying pinned Quarto version to other workflows

* Removing test notebook

* docs: Workflow Wait step time delta configuration (#954)

* Setup: Wait step time delta

* Editing

* Updating example screencap for Wait

* Remove login button and make related CSS modular (#949)

* Remove login button and make related CSS modular

* Make login link specific to profile

* Change scss/ to assets/style/ and update README.md

* Fix double space

* Check out .gitignore from main

* Minor README.md tweak

* One more minor README.md tweak

* Add missing backticks

* Fix Log In button for developer.css

* Minor edits

* Slight adjust to CSS in-progress message

---------

Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>

* docs: Updated "Manage model finding fields" (#958)

* Edit: DElete model findings

* Edit: Manage model finding fields

* Editing UI wording

* Editing...

* Editing......

* Proofreading

* Edit

* docs: Updated "Manage document templates" (#960)

* Draft: Delete document templates

* Edit: View document templates

* Wrapping up

* Stupid broken link

* Edit: VDV Findings > Observations (#962)

* docs: Update inventory & finding field views (#964)

* Edit: Manage model inventory fields

* Save point

* Edit: Manage model finding fields

* Cleanup

* Proofread

* Adjustments

* Final touch

* Fix bug in Makefile (#935)

* Bug bash: nrichers/sc 12201/disable algolia search on api reference pages (#968)

* Disable Algolia search on Public API reference page

* Fix stray comment indent

* docs: Add shared text blocks to YAML templates (#970)

* Setup: Add library blocks to YAML templates

* Save point

* Save point

* Tidying up

* docs & training: Virtual Document Validator, regulations & policies set up update (#973)

* Draft: Questions from PDF uploads

* Training: Setup for regulations & policies

* Save point

* Save point

* Draft: Administrator Fundamentals update

* Draft: Developer Fundamentals update

* Proofreading

* Hiding PDF upload for VDV (#975)

* Add system access requirements (#977)

* Add initial system access requirements topic

* Text edits

* More edits

* Fix incomplete sentence

* Basic version: Improve navigation for Python API and Public REST API (#966)

* Apply the basic fixes we agreed on

* Remove a:hover experiment

* Update .gitignore

* Underline specific links only

* Address review comments from Beck

* Final CSS tweak

* Update release_notes_check.py (#981)

* Enable release notes checks to post comment with instructions (#984)

* Update release_notes_check.yaml

* Update release_notes_check.py

* Edit: Remove info for add finding via doc section (#978)

* docs: Updated Workflows Approval Step info (#979)

* Edit: Approval step & condition instructions

* Update: Approval Step example

* Add Tidy Layout callout

* Bump versions to 25.10 & v2.9.5 + public release landing page improvements (#971)

* Bump release versions

* Add temporary branch ref

* Fix Makefile help

* Reorder all releases page, bump up length of descriptions shown

* Turn older releases into collapsible section

* Fix YAML

* Fix YAML again

* Add pagination style from release notes portal

* Add button for all feature highlights & add whitespace

* Wording tweak

* Run make get-source

* All releases page tweaks

* Fix broken links

* Change heading to Latest features for consistency

* Address Beck's sidebar suggestion

* Fix date in this repo, too

* Add whitespace before heading

* Remove the branch ref I had THREE reminders for to remove ... (#991)

---------

Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
nrichers added a commit that referenced this pull request Oct 31, 2025
* Removing validmind version variable (#921)

* infra: Merge main into staging PRs created by staging deploy now indicate original PR# (#924)

* Updating main into staging workflow

* Testing branch versions

* Removing test file

* Fix Makefile help

* infra: Delete older PR preview comments from validate & execute flows (#932)

* Adding deletion of old preview comments back in

* Removing unused variable

* Adding same thing to notebooks

* Edit to notebook, should remove old comment

* hm

* Deleting test execution notebook

* ValidMind Platform releases: Exclude `admin-ui` from roundup listing page (#933)

* Exclude admin-ui from platform releases roundup

* Adjust

* Remove old variable usage

---------

Co-authored-by: Nik Richers <nik@validmind.ai>

* docs: Export findings (#939)

* Setup: Customize model findings layout

* Save point

* Save point

* Draft: Customize model finding layouts

* Setup: Export findings

* Draft: Export findings

* Export documentation > Export documents

* Editing

* Cleanup

* Proofread

* Filter

* Update API reference to use prod server (#938)

* Update API reference to use prod server

* Update Makefile for admin API as well

* docs: Export inventory (#943)

* Setup: Export inventory

* Draft: Export inventory

* Proofreading

* Oops

* docs: Updated Export findings (#944)

* Setup

* Proofread

* Testing pinned Quarto version

* docs: Customize Virtual Document Validator (#947)

* Set up: Manage regulations and policies

* WIP

* Customize VDV draft

* Edits: Check documentation for compliance

* Proofread

* Quick UI updates

* Pin a Quarto version specified in Action variables for our deployment workflows (#951)

* Testing pinned Quarto version

* Applying pinned Quarto version to other workflows

* Removing test notebook

* docs: Workflow Wait step time delta configuration (#954)

* Setup: Wait step time delta

* Editing

* Updating example screencap for Wait

* Remove login button and make related CSS modular (#949)

* Remove login button and make related CSS modular

* Make login link specific to profile

* Change scss/ to assets/style/ and update README.md

* Fix double space

* Check out .gitignore from main

* Minor README.md tweak

* One more minor README.md tweak

* Add missing backticks

* Fix Log In button for developer.css

* Minor edits

* Slight adjust to CSS in-progress message

---------

Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>

* docs: Updated "Manage model finding fields" (#958)

* Edit: DElete model findings

* Edit: Manage model finding fields

* Editing UI wording

* Editing...

* Editing......

* Proofreading

* Edit

* docs: Updated "Manage document templates" (#960)

* Draft: Delete document templates

* Edit: View document templates

* Wrapping up

* Stupid broken link

* Edit: VDV Findings > Observations (#962)

* docs: Update inventory & finding field views (#964)

* Edit: Manage model inventory fields

* Save point

* Edit: Manage model finding fields

* Cleanup

* Proofread

* Adjustments

* Final touch

* Fix bug in Makefile (#935)

* Bug bash: nrichers/sc 12201/disable algolia search on api reference pages (#968)

* Disable Algolia search on Public API reference page

* Fix stray comment indent

* docs: Add shared text blocks to YAML templates (#970)

* Setup: Add library blocks to YAML templates

* Save point

* Save point

* Tidying up

* docs & training: Virtual Document Validator, regulations & policies set up update (#973)

* Draft: Questions from PDF uploads

* Training: Setup for regulations & policies

* Save point

* Save point

* Draft: Administrator Fundamentals update

* Draft: Developer Fundamentals update

* Proofreading

* Hiding PDF upload for VDV (#975)

* Add system access requirements (#977)

* Add initial system access requirements topic

* Text edits

* More edits

* Fix incomplete sentence

* Basic version: Improve navigation for Python API and Public REST API (#966)

* Apply the basic fixes we agreed on

* Remove a:hover experiment

* Update .gitignore

* Underline specific links only

* Address review comments from Beck

* Final CSS tweak

* Update release_notes_check.py (#981)

* Enable release notes checks to post comment with instructions (#984)

* Update release_notes_check.yaml

* Update release_notes_check.py

* Edit: Remove info for add finding via doc section (#978)

* docs: Updated Workflows Approval Step info (#979)

* Edit: Approval step & condition instructions

* Update: Approval Step example

* Add Tidy Layout callout

* Bump versions to 25.10 & v2.9.5 + public release landing page improvements (#971)

* Bump release versions

* Add temporary branch ref

* Fix Makefile help

* Reorder all releases page, bump up length of descriptions shown

* Turn older releases into collapsible section

* Fix YAML

* Fix YAML again

* Add pagination style from release notes portal

* Add button for all feature highlights & add whitespace

* Wording tweak

* Run make get-source

* All releases page tweaks

* Fix broken links

* Change heading to Latest features for consistency

* Address Beck's sidebar suggestion

* Fix date in this repo, too

* Add whitespace before heading

* Remove the branch ref I had THREE reminders for to remove ... (#991)

* docs & training: Calculation Finding Fields (#988)

* Draft: Calculation Finding Fields

* Draft: Calculation Finding Fields in training

* Save point

* Cleaning up

* Proofreading

* Save point

* Update site/guide/model-inventory/_field-types.qmd

Co-authored-by: Nik Richers <nik@validmind.ai>

* Update _field-types.qmd

* Applying Andres's code edit

---------

Co-authored-by: Nik Richers <nik@validmind.ai>

* Upgrade Tachyons with official codeblock fix & remove our workaround (#995)

* Commit regenerated Python API files and bump version (#997)

* docs: Workflows for findings (#998)

* Setup: Workflows for findings

* Renaming Setting up model workflows page

* Fixing links

* Sidebar link

* Renaming directory

* Fixing broken links pt2

* Cleaning up...

* More cleanup

* More

* More renaming

* Oops

* Almost there

* Stragglers

* One more?

* & includes

* Save point

* Edit: Add new workflows in docs

* Edit: Edit existing workflows in docs

* Edit: Review active workflows in docs

* Edit: Review workflow histories in docs

* Save point

* Edit: Initiate workflows in docs

* Edit: Abort workflows in docs

* Edit: Manage workflow tasks

* Edit: Transition workflows

* More cleaup

* Training: Edit Add new workflows

* Training: Review active workflows

* Training: Review workflow history

* Training: Initiate workflows

* Training: Reset workflows

* Training: Abort workflows

* Training: Transition workflows

* Proofreading

* More proofreading

* ooo figured out a neat conditional hack

* Editing training slides

* Updating single-source templates

* docs & training: Updates to model registration UX (#1003)

* Edit: Register models in inventory

* Quickstart: Register model developer

* Quickstart: Register model validator

* Updating code snippet images

* Updating developer model details image

* Updating validation report details image

* Admin training: Register demo model

* Developer training: Register demo model

* Validator training: Register demo model

* Cleanup

* Proofreading

* Edit: Manage model stakeholder types on registration

* Edit: Manage model stakeholder types on registration training pt1

* Edit: Manage model stakeholder types on registration training pt2

* Edit

* Edit

* Pulling in latest notebooks from sibling PR

* Nrichers/sc 12743/fix quarto docs workflow issue (#1001)

* Remove validmind/ folder and old python-docs.zip file

* Change make python-docs to generate the Python API files

* Add site/validmind to the .gitignore

* Add make python-docs equivalent to workflows

* Try RELEASE_NOTES_RO_PAT

* docs: Workflows User Action v2 step deadlines (#1005)

* Setup: Edits to User Action v2 deadlines

* Save point

* Save point

* Updated example images

* Proofread

* Formatting

* Explicitly install poetry before generating Python API docs

---------

Co-authored-by: Nik Richers <nik@validmind.ai>

* Setup: Trigger workflow start on date & datetime (#1008)

* Remove Slack sign-up page & update community resources (#1010)

* Remove Slack sign-up page & update community resources

* Minor consistency edit

* Remove sign-up hint I added

* docs: Manage finding severities (#1013)

* Setup: Manage finding severities

* Docs: Manage finding severities

* Training: Manage finding severities

* Cleaning up community page

---------

Co-authored-by: Beck <164545837+validbeck@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Not to be externalized in the release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants